Skip to content

Conversation

@tejasbadadare
Copy link
Contributor

Summary

  • Prevent all updates to permanent subs since any kind of update can be used to harm the availability of the feed
  • Remove GasConfig. Gas limits will be set on the Keeper for simplicity and we will take the risk on spiking gas costs.

How has this been tested?

  • Current tests cover my changes
    • Updated existing tests
  • Added new tests
  • Manually tested the code

@vercel
Copy link

vercel bot commented Apr 22, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

6 Skipped Deployments
Name Status Preview Comments Updated (UTC)
api-reference ⬜️ Ignored (Inspect) Visit Preview Apr 22, 2025 9:05pm
component-library ⬜️ Ignored (Inspect) Visit Preview Apr 22, 2025 9:05pm
entropy-debugger ⬜️ Ignored (Inspect) Visit Preview Apr 22, 2025 9:05pm
insights ⬜️ Ignored (Inspect) Visit Preview Apr 22, 2025 9:05pm
proposals ⬜️ Ignored (Inspect) Visit Preview Apr 22, 2025 9:05pm
staking ⬜️ Ignored (Inspect) Visit Preview Apr 22, 2025 9:05pm

@tejasbadadare tejasbadadare merged commit 1451636 into main Apr 23, 2025
10 checks passed
@tejasbadadare tejasbadadare deleted the tb/pulse-scheduler/permanent-sub-freeze-config branch April 23, 2025 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants